n8n/nodes/Tinyfish/GenericFunctions.ts (1)

223-225: Redundant try/catch that only rethrows.

The outer catch block on lines 223-225 catches the error and immediately rethrows it without any transformation or logging. This is unnecessary since the inner error handling (lines 147-154) already throws appropriate errors.

♻️ Proposed fix to remove redundant wrapper

 export async function consumeSseStream(
 	this: IExecuteFunctions,
 	payload: IDataObject,
 ): Promise<IDataObject> {
 	const credentials = await this.getCredentials('tinyfishApi');
 	const apiKey = credentials.apiKey as string;

 	let lastProgress = '';

-	try {
-		const response = await fetch(`${API_BASE_URL}/v1/automation/run-sse`, {
+	const response = await fetch(`${API_BASE_URL}/v1/automation/run-sse`, {
 		method: 'POST',
 		headers: {
 			'X-API-Key': apiKey,
 			'Content-Type': 'application/json',
 		},
 		body: JSON.stringify(payload),
-		});
+	});
 
 		// ... rest of the function body unchanged ...
 
 		return finalResult;
-	} catch (error) {
-		throw error;
-	}
 }

🤖 Prompt for AI Agents

Verify each finding against the current code and only fix it if needed.

In `@n8n/nodes/Tinyfish/GenericFunctions.ts` around lines 223 - 225, The outer
try/catch that catches and immediately rethrows the error is redundant—remove
the catch block (the closing catch { throw error; } that follows the surrounding
try in GenericFunctions.ts) so errors from the inner logic (which already throw
appropriate errors around the inner error handling) propagate naturally; ensure
you only delete the catch clause and its brace while keeping the try body intact
so compilation and control flow are unchanged.